Skip to content

Comments

week2 HW by FennyLiang#7

Open
FennyLiang wants to merge 15 commits intoagileworks-tw:masterfrom
FennyLiang:master
Open

week2 HW by FennyLiang#7
FennyLiang wants to merge 15 commits intoagileworks-tw:masterfrom
FennyLiang:master

Conversation

@FennyLiang
Copy link

first pr

- Add Facebook id to store Facebook id.
- Change property's name.
* feature/improve_test_case:
  Change name.
  Fix test for find.
  - Fix code style. - Change property's name.
  Fix syntax.
  Map api response to our model's format manually.
  Add test to ensure fetch friends success. (However, some people have no friends, that is not issue in this case)
  Prepare friends data at before to fit language mean.
  Change global property's name.
  - Fix code style. - Add Facebook id to store Facebook id.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant